Skip to content

Conversation

@n-berenice
Copy link

Minor changes to grammar. I might be able to edit the smartphone screenshots as well but getting the fonts and alignment right is proving to be difficult as they're in .png instead of layered XCF.

& doesn't exists in Spanish
Adjectives are mostly postpositive in the Spanish syntax.
The use of "el" would imply Hiragana and Katakana are one and the same alphabet when they're not, so "los" is better when talking about two or more different things.
The "Ni" conjunction is better suited for the sentence.
i18n/es.json refers to Hiragana and Katakana as syllabaries, this is correct as they're not alphabets.
Copy link
Owner

@aeri aeri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ironically the Spanish translation is not as it should be, thanks for the corrections.

Although the character "ampersand" is a non-alphabetizable symbol in the Spanish alphabet, its use in the name of the application is purely for aesthetic purposes.

Do you propose also translate the images?

@sakci
Copy link
Contributor

sakci commented Jan 4, 2022

Do you propose also translate the images?

I think it would be better if we gave you the translation of the images and you edit it yourself, since:

  • We don't know the name of the font you're using.
  • Phone screenshots should also be replaced with the localized ones and it'd be cumbersome for us to do it. First of all, we don't have the image template. Second, we'd have to compile the app ourselves just for the screenshots. You'll compile a new version for the update anyway so it'd be better if you handle it.

@n-berenice
Copy link
Author

Agree with sakci, I tried to translate the images but they're too complex to edit the easy way, the backgrounds aren't solid meaning if I have to cut something I'll end up with a white rectangle with no background.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants